Skip to content

Conversation

topecongiro
Copy link
Contributor

@topecongiro topecongiro commented Dec 28, 2017

Closes #29924. Closes #38857. Closes #39665. Closes #39872.
Closes #41210. Closes #41880. Closes #43483.

Note that compile-fail/E0599.rs is for #41210.

@rust-highfive
Copy link
Contributor

r? @aturon

(rust_highfive has picked a reviewer for you, use r? to override)

}

impl Trait {
//~^ ERROR E0038
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you use actual error messages instead of codes outside of E0___.rs tests?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your review. Updated and rebased.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@topecongiro could you do that change in all compile-fail files?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@estebank Update tests and reabased. Also fixed the issue number (#39953 -> #39553).

@kennytm kennytm added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Dec 28, 2017
@estebank
Copy link
Contributor

#39953 is a PR, removed from description.

@estebank
Copy link
Contributor

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Dec 31, 2017

📌 Commit b3c022d has been approved by estebank

@bors
Copy link
Collaborator

bors commented Dec 31, 2017

⌛ Testing commit b3c022d with merge 2d74ce2...

bors added a commit that referenced this pull request Dec 31, 2017
Add tests on fixed ICEs

Closes #29924. Closes #38857. Closes #39665. Closes #39872.
Closes #41210. Closes #41880. Closes #43483.

Note that compile-fail/E0599.rs is for #41210.
@bors
Copy link
Collaborator

bors commented Dec 31, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: estebank
Pushing 2d74ce2 to master...

@bors bors merged commit b3c022d into rust-lang:master Dec 31, 2017
@topecongiro topecongiro deleted the fixed-ices branch December 31, 2017 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants